Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Poc bedrock 3 #2736

Draft
wants to merge 35 commits into
base: poc-bedrock
Choose a base branch
from
Draft

Poc bedrock 3 #2736

wants to merge 35 commits into from

Conversation

rauboti
Copy link
Contributor

@rauboti rauboti commented Nov 27, 2024

Draft PR for POC 3. PR dekker funksjonalitet fra steg 3 som skal:

  • Generere alt tekst for bilde
  • Transkribere lyd til tekst

@gunnarvelle
Copy link
Member

Generer-knappen bør vel helst befinne seg i bildeskjemaet slik at det kan genereres ved oppretttelse av bilde. Skader ikkje å ha det på bilderedigering i artikkel også, men då som en tilleggsfunksjon.

@gunnarvelle
Copy link
Member

Får ikkje til å generere alt-tekst på bilder fra bilde-skjemaet. Server-endepunktet trigges ikkje.

src/server/api.ts Outdated Show resolved Hide resolved
@gunnarvelle
Copy link
Member

Sliter litt med å teste dette siden eg må kjøre mot lokal audio-api og då finner eg ingen audio-filer.

@gunnarvelle
Copy link
Member

Sliter litt med å teste dette siden eg må kjøre mot lokal audio-api og då finner eg ingen audio-filer.

Fant ut av det. Det var bøtta med lokale filer som ikkje hadde tilgangskontroll satt opp korrekt

@gunnarvelle
Copy link
Member

Det skjer ingen post når eg skal transkribere audio. Noko eg gjør feil? Har satt miljøvariabler så vidt eg kan sjå.
Ser i lokal audio-api følgende: GET /audio-api/v1/transcription/audio/69/no-NO executed in 287ms with code 404

@gunnarvelle
Copy link
Member

Det gjøres en sjekk på om jobben er i gang, men om du kaller denne adressa første gang får du 404. Så eit eller anna som må fikses her.

@gunnarvelle
Copy link
Member

Du må legge inn vercel som devdep for at pr-installasjon skal lages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants